Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add branch 18.0 (no default branch change) #41

Merged
merged 1 commit into from
Oct 8, 2024

Conversation

simahawk
Copy link
Contributor

@simahawk simahawk commented Oct 7, 2024

Possible replacement for #36

conf/repo/social.yml Show resolved Hide resolved
@simahawk
Copy link
Contributor Author

simahawk commented Oct 7, 2024

@sbidoul any blocker for the merge?

@sbidoul
Copy link
Member

sbidoul commented Oct 7, 2024

I was wondering if we could test on a couple more repos, to save me a mass update in case something is missing in the template, but otherwise, good for me.

@simahawk
Copy link
Contributor Author

simahawk commented Oct 7, 2024

@sbidoul we have some WIP on v18 already. I'll open another PR w/ a shortlist of repos (eg: queue, connector, etc)

@sbidoul
Copy link
Member

sbidoul commented Oct 7, 2024

We are ready with fs_storage and session_db, so we can do #42 too

@simahawk
Copy link
Contributor Author

simahawk commented Oct 7, 2024

My subset here #43

@simahawk simahawk force-pushed the add-v18-no-default branch from a8fd695 to 54f0d32 Compare October 7, 2024 09:11
@simahawk
Copy link
Contributor Author

simahawk commented Oct 7, 2024

IMO this is working pretty well. I don't see any other reason to wait. Do you?

@simahawk
Copy link
Contributor Author

simahawk commented Oct 8, 2024

@sbidoul approved :)

@simahawk simahawk merged commit 5486888 into OCA:master Oct 8, 2024
1 check passed
@simahawk simahawk deleted the add-v18-no-default branch October 8, 2024 07:10
@simahawk
Copy link
Contributor Author

simahawk commented Oct 8, 2024

Looks like we have an issue #44

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants